Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Traduz o arquivo 2013-05-10-EYDeploy.markdown #37

Merged
merged 4 commits into from
Dec 10, 2017

Conversation

leticia
Copy link
Contributor

@leticia leticia commented Oct 15, 2017

Não traduzi alguns textos de botão e seções referentes ao site da Engine Yard porque achei que talvez seja mais fácil de encontrá-los em seu texto original, mas se gor melhor traduzir, faço as alterações! 👍

Ah, eu não traduzi o permalink, qual tem sido o padrão?


Go to Engine Yard's [Website](https://www.engineyard.com/) and click on "GET STARTED FREE" to sign up for your Free 500 Hour Trial. The sign up will send you a confirmation email, so go check your email, click on the link and sign in to your account. Click on the link under "Your available applications" that says "Engine Yard Cloud". This will take you to your dashboard.
Navegue até o [site](https://www.engineyard.com/) da Engine Yard e clique em "GET STARTED FREE" para obter suas 500 horas de teste grátis. O registro lhe enviará um e-mail de confirmação; verifique seu e-mail, clique no link e entre em sua conta. Clique no link sob "Your available applications" que diz "Engine Yard Cloud". Isto leva ao seu painel.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acho que nas partes em inglês vc pode deixar a tradução em parenteses tipo "GET STARTED FREE" (inicie gratuitamente)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mesma coisa para "Your available applications"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa, brigada! Vou fazer essas alterações. Tu achas que vale também traduzir o permalink da página?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acho que não precisa... Não estamos mexendo nos permalinks nos outros PRs (pelo que eu tenha visto)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, @bahbbc! Veja se ficou melhor assim.

@anaschwendler
Copy link
Member

Gente, que honra revisar o PR de alguém que me introduziu ao Rails Girls, @leticia vou revisar seu PR tanto de estilos quanto português, acho que já foi revisado, mas vambora:

O que eu fiz para testar seu PR:

  1. Clonei o guia:
$ git clone git@github.com:railsgirls/guides-ptbr.git
  1. Entrei na pasta do projeto:
$ cd guides-ptbr
  1. Rodei o ambiente:
$ bundle install
  1. Fiz checkout para a branch a ser testada:
$ git checkout -b leticia-issue-2 master
  1. Adiciono ao meu projeto local, o conteúdo da branch:
$ git pull https://github.com/leticia/guides-ptbr.git issue-2
  1. Rodei localmente o projeto:
$ bundle exec jekyll server
  1. Acessei http://localhost:4000/engineyard e o resultado:
    image

Tenho algumas sugestões de melhora :)

Copy link
Member

@anaschwendler anaschwendler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Era isso aí, tenho algumas sugestões pra melhorar o tutorial ainda mais :)


*Created by Mary Jenn, [@mfjenn](https://twitter.com/mfjenn)*
*Criado por Mary Jenn, [@mfjenn](https://twitter.com/mfjenn)*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Você traduziu, se dê os créditos! Geralmente a gente coloca:
image

You'll need to get your database to work on Engine Yard, which uses a different database than the Rails default. Please change the following in the Gemfile:
Você precisará fazer com que seu banco de dados funcione no Engine Yard, que
utiliza um banco diferente do padrão do Rails. Por favor, substitua o seguinte
código no arquivo `Gemfile`:


gem 'sqlite3'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aqui a gente precisa de uma melhora, sei que não está no original, mas estou corrigindo conforme vejo:

Essa parte do código, geralmente vem com uma sintaxe de highlight, daí sugiro que pra ficar padrãozinho, seja melhor colocar das linhas 17 - 19:

{% highlight ruby %}
gem 'sqlite3'
{% endhighlight %}

Para ficar assim:
image



gem 'sqlite3'


to
para

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aqui também, sugestão:

{% highlight ruby %}
group :development do
  gem 'sqlite3'
end


group :production do
  gem 'pg'
end
{% endhighlight %}

Das linhas 24 - 33, fica assim:
image


You need to add your app to your Git repository. You can do this by running the following in the terminal:
Você precisa adicionar sua aplicação ao seu repositório Git. Você pode fazer
isso executando o seguinte no seu terminal:


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comandos git estamos adicionando assim:

Linhas 44 - 54:

{% highlight sh %}
git init
{% endhighlight %}

{% highlight sh %}
git add .
{% endhighlight %}

{% highlight sh %}
git commit -m "initial commit"
{% endhighlight %}

Ficando assim:
image

@marinalimeira
Copy link
Member

Oi @anaschwendler, adicionei tuas correções aqui e to mergeando :)

@marinalimeira marinalimeira merged commit 7f87981 into railsgirls:master Dec 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants